Skip to content

Fix typo#3247

Closed
milosa wants to merge 1 commit into
docker:masterfrom
milosa:patch-1
Closed

Fix typo#3247
milosa wants to merge 1 commit into
docker:masterfrom
milosa:patch-1

Conversation

@milosa

@milosa milosa commented Aug 12, 2021

Copy link
Copy Markdown

No description provided.

@milosa milosa requested a review from thaJeztah as a code owner August 12, 2021 13:33
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #3247 (e59f622) into master (b4dde3a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3247   +/-   ##
=======================================
  Coverage   57.99%   57.99%           
=======================================
  Files         302      302           
  Lines       21748    21748           
=======================================
  Hits        12613    12613           
  Misses       8212     8212           
  Partials      923      923           

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The change itself looks OK, but I see the commit is missing a DCO sign-off, which makes CI fail the failure shows some instructions on fixing that (unfortunately, it requires the commit to be amended on the CLI, and cannot be done through the GitHub web UI); https://github.com/docker/cli/pull/3247/checks?check_run_id=3312135659

When signing-off, make sure to use your real name (not the GitHub handle)
https://github.com/docker/cli/blob/master/CONTRIBUTING.md#sign-your-work

Let me know if you need help!

@thaJeztah

Copy link
Copy Markdown
Member

closing as this was addressed in 6fd4cff / #4712

@thaJeztah thaJeztah closed this May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants